-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merge with distutils@8c3c3d29 #3809
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Because most Python packages maintain tests in top-level directory `tests/` instead of `test/`, add both to source distributions for consistency.
C99 removed support for implicit function declarations. This means that just calling a function, without declaring the function first, can result in a compilation error. Today, has_function works with most compilers because they issue just a warning, create an object file, and attempt a link, which then detects available of the symbol at link time, as intended. With future compilers, compilation will already fail, and no link test is performed. The has_function interface provides the caller with a way to supply a list of header files to include. However, even with today's compilers, this only works if the function does not expect any parameters. Otherwise, the function call in the C fragment created by has_function will not supply the correct argument list and fail compilation. Therefore, this change supplies and incorrect prototype without arguments. This is what autoconf does today in a very similar situation, so it is quite likely that compilers will support this construct in this context in the future. The includes and include_dirs arguments are deprecated because of the parameter list mismatch issue. Fixes #3648.
[FURB123]: Replace `str(x)` with `x`
[FURB113]: Use `x.extend(...)` instead of repeatedly calling `x.append()`
[FURB108]: Use `x in (y, z)` instead of `x == y or x == z`
[FURB131]: Replace `del x[y]` with `x.pop(y)`
[FURB109]: Replace `in [x, y, z]` with `in (x, y, z)`
Link directly to PEP
Apply refurb suggestions
Update outdated GitHub Actions
For 'get_python_inc', bypass missing config include dir
Fix MinGW-w64 segmentation fault
Fix typos found by codespell
Add `tests/test*.py` to source distributions
distutils.ccompiler: Make has_function work with more C99 compilers
The previous code did not take into account the .exe file extension on Windows.
Fix MinGW-w64 segmentation fault (v2)
distutils.ccompiler: Remove correct executable file on Windows
add a pypy CI run
Fixed accumulating include dirs after compile
jaraco
force-pushed
the
distutils-8c3c3d29
branch
from
February 7, 2023 01:09
c7f3834
to
47c7cfd
Compare
2 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
tests/test*.py
to source distributionsSummary of changes
Closes
Pull Request Checklist
changelog.d/
.(See documentation for details)